Skip to content

+Restored public interface to post_data_1d_k#169

Merged
marshallward merged 1 commit into
NOAA-GFDL:dev/gfdlfrom
Hallberg-NOAA:restore_post_data_1d_k
Jul 18, 2022
Merged

+Restored public interface to post_data_1d_k#169
marshallward merged 1 commit into
NOAA-GFDL:dev/gfdlfrom
Hallberg-NOAA:restore_post_data_1d_k

Conversation

@Hallberg-NOAA
Copy link
Copy Markdown
Member

Made post_data_1d_k publicly visible once again, rather than requiring calls
to use post_data, to support backward compatibility with older versions of the
ocean_BGC code. This interface was removed from public visibility as a part of
github.com//pull/107, but it caused problems with some of GFDL's
Earth System Models, as noted in #168.
All answers are bitwise identical for any cases that compiled before.

  Made post_data_1d_k publicly visible once again, rather than requiring calls
to use post_data, to support backward compatibility with older versions of the
ocean_BGC code.  This interface was removed from public visibility as a part of
github.com/NOAA-GFDL/pull/107, but it caused problems with some of GFDL's
Earth System Models, as noted in NOAA-GFDL#168.
All answers are bitwise identical for any cases that compiled before.
@Hallberg-NOAA Hallberg-NOAA linked an issue Jul 18, 2022 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 18, 2022

Codecov Report

Merging #169 (9e3735c) into dev/gfdl (64fe4fc) will not change coverage.
The diff coverage is n/a.

❗ Current head 9e3735c differs from pull request most recent head fb679b5. Consider uploading reports for the commit fb679b5 to get more accurate results

@@            Coverage Diff            @@
##           dev/gfdl     #169   +/-   ##
=========================================
  Coverage     33.99%   33.99%           
=========================================
  Files           259      259           
  Lines         70248    70248           
  Branches      13016    13016           
=========================================
  Hits          23884    23884           
  Misses        41865    41865           
  Partials       4499     4499           
Impacted Files Coverage Δ
src/framework/MOM_diag_mediator.F90 53.02% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64fe4fc...fb679b5. Read the comment docs.

@marshallward
Copy link
Copy Markdown
Member

Gaea regression: https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/16148 ✔️

@marshallward marshallward merged commit 2217e63 into NOAA-GFDL:dev/gfdl Jul 18, 2022
@Hallberg-NOAA Hallberg-NOAA deleted the restore_post_data_1d_k branch February 2, 2023 13:26
marshallward pushed a commit that referenced this pull request Mar 12, 2026
update MOM6 to its main repo. 20260112 updating
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Removal of public post_data_1d_k is incompatible with ocean_BGC

2 participants